Skip to content

Performer select calculated ages#5110

Merged
WithoutPants merged 10 commits intostashapp:developfrom
dogwithakeyboard:performer-select-calculated-ages
Mar 5, 2025
Merged

Performer select calculated ages#5110
WithoutPants merged 10 commits intostashapp:developfrom
dogwithakeyboard:performer-select-calculated-ages

Conversation

@dogwithakeyboard
Copy link
Contributor

@dogwithakeyboard dogwithakeyboard commented Aug 2, 2024

Following on from #5076 this shows the age of the performer in the select based on the scene/image/gallery date, when one exists.

The width of the performer select in the tagger view has also been slightly increased.

perfage3

@DogmaDragon
Copy link
Collaborator

Censored the image for safety reasons.

@echo6ix
Copy link
Contributor

echo6ix commented Aug 3, 2024

From what I can tell it looks like the $age value is being inserted into the CSS birthdate selector. The age value should get its own CSS selector.

<span className="performer-select-age">{$age}</span>

@stashapp stashapp deleted a comment Aug 4, 2024
@stashapp stashapp deleted a comment Aug 4, 2024
@echo6ix
Copy link
Contributor

echo6ix commented Aug 7, 2024

@kermieisinthehouse @dogwithakeyboard @WithoutPants

image

This is very nitpicky, and I do not want to stifle this PR, but the proposed locale string {age} on this date comes directly after the the birthdate (yyyy-mm-dd) value. See above image.

To you and I, we know which date is being referenced, but on it's face as far as clear and unambiguous language is concerned, this is really awkward phrasing as someone who's not familiar with the app could misconstrue what "on this date" is referencing, especially being juxtaposed to the birthdate .

To remove ambiguity I would suggest {age} {years old} on scene date and {age} {years old} on image date respectively. It goes back to referencing the context, but in a way that does not imply they were in the content.

Does that make sense?

The Performer card had "x years old in this scene", regardless of what sort of media it was attached to. I have made both strings "x [years old] at production instead.
@WithoutPants
Copy link
Collaborator

To remove ambiguity I would suggest {age} {years old} on scene date and {age} {years old} on image date respectively. It goes back to referencing the context, but in a way that does not imply they were in the content.

In order to move this forward, I've changed the wording on this to be x at production, which is concise, clear and unambiguous to me.

The wording in the performer card in a scene/image/gallery was also incorrect for image or gallery - x years old in this scene. I have changed this to x years old at production. These don't necessarily have to be the final wordings, but that can refined in a separate PR.

@WithoutPants WithoutPants added the improvement Something needed tweaking. label Feb 28, 2025
@WithoutPants WithoutPants added this to the Version 0.28.0 milestone Feb 28, 2025
@WithoutPants WithoutPants merged commit df55667 into stashapp:develop Mar 5, 2025
2 checks passed
XGFan pushed a commit to XGFan/stash that referenced this pull request Mar 27, 2025
* Change wording of performer age at production

The Performer card had "x years old in this scene", regardless of what sort of media it was attached to. I have made both strings "x [years old] at production instead.
---------
Co-authored-by: WithoutPants <53250216+WithoutPants@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Something needed tweaking.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants